-
Notifications
You must be signed in to change notification settings - Fork 922
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Redesign #498
Redesign #498
Conversation
I've put a test site up at http://redesign.apis.dev.openstreetmap.org/ though note that it's a bit slow at the moment due to network issues at the hosting site. |
Is the plan to do this as one large pull or as independent pulls? Also, do you want broad-level feedback, or specific issues like the 1px gap in |
Looks great. Probably the most significant issue to my eyes is content width on large screens: at (say) 1100px, /welcome looks pretty uncomfortable, and could maybe work better as a centred 700px column. Couple of little details on browsing around:
|
Suggestions:
|
Ah, now I see it's on the todo. |
If #456 gets merged separately I'll rebase this. The rest will probably be one pull.
Right now, broad-level and missing/broken functionality, so I can complete the TODO. Wait a bit for cosmetic issues. |
Will there be a way to "close" the search result view (or history, export or data view)? What is the disabled |
Enable then disable the data layer: the sidebar still displays "data". Alas, no objects are displayed: it seems like the data layer is broken. |
Others have pointed out the problem of the topbar on small screens / windows not fiting horizontaly. But I find that the extra vertical space wastes valuable space, clashes with iD (or is there a pull request pending for iD too ?), and is simply not as good loking as the original. I'm not sure either what is the criteria for buttonizing some links and not others; I'd be happy with no button in sight. I like seeing "Help" and "User Diaries" in there, but I'm not so sure about "GPS Traces" and "About" being at such a prominent place. They're just not visited often enough (IMHO, but our sysadmins may have the stats) to warrant this position on the page. Which brings me to the sidebar... It has been emptied of most of its content and made larger at the same time. It's great to use it for search results, a welcome improvement. But to me it could still be used for the old content, such as "about". Also, displaying search result hides the "make donation", "where am I" and other items from the sidebar, without a visible way to get back to the original. I suggest that displaying search results be done in a dedicated part of the sidebar, without disturbing the rest of the content. |
Some points:
|
I too would like to see traces display on the map, but that needs to be a separate change. This one is already too big. :) Regarding "Export" and other navigation elements -- I think they're the best choice for now. The organizing rationale is that "View", "History", and "Export" all have a "primary" feel based on a common sidebar+map layout, and "GPS Traces", "User Diaries", etc have a content-only layout and "secondary" feel, so grouping them that way in the header makes sense. Eventually I do think we'll need to develop more of an information hierarchy around higher-level concepts like "Data" and "Community", and nest specific pages like "Export" and "History" underneath. A map-based "GPS Traces" page would fit better that way too. But for now, the design is intentionally closer to the existing organization, so that we can actually accomplish some incremental change. Given how much this changes the navigation menus, a responsive small screen design will necessarily be a part of it -- that's on the TODO list. |
It might be worth splitting this PR into multiple ones. At least separate the new sidebar behavior (collapsible and containing the search results) from changing the links locations and styling. I think the former is less controversial than the later, and could be merged more easily ? Concerning the sidebar, I still see no way to get back to the previous state after having searched for something or closed the sidebar. Beyond having the search result not completely replacing the original sidebar content, I'm not sure what a good UI would be. |
Some thoughts:
|
I use the changeset views more than most people probably do, so I'm going to start there Issues
Questions
|
#zoom/lat/lon hash format is now used consistently.
This has been merged - note that I rewrote the branch to fix @edenh's email address so the hashes will all be different. It hasn't been deployed yet - will probably wait until the weekend to let people get started on translations. Have pinged Siebrand about getting Translatewiki updated. |
The copyright and license is an important part of OpenStreetMap. This restores the copyright & license link to the menu, placing it in the top-right with the other pages which do not display a map. Fixes openstreetmap#548 Fixes comments in openstreetmap#498 (comment)
(I'm assuming that what I'm seeing on the api06 server is what is intended to be deployed) Changeset display draws attention to table information that tends not to vary ("created_by" etc.) and not to the information that does (comments, relations, ways, nodes). Here's the a new one: http://api06.dev.openstreetmap.org/changeset/35139 and here's what the site does now: http://www.openstreetmap.org/browse/changeset/18722658 (actually, more readable still was the previous version of "changeset display" which displayed just the important information without the extra "pretty" cruft). BTW - I was amused to see the "just a list of numbers" comment about the list of nodes higher up in this list. That's exactly what a changeset is, and that's the information that's useful! |
@SomeoneElseOSM the "numbers" may be what the changeset is about, but what does know that a changeset contains node 3452353456 actually tell you? do you remember the IDs of all the nodes you are interested in? |
@tomhughes If you're reviewing changesets by new mappers, the number tells you whether it was an old node that's been changed in some way or one that's been newly created and perhaps edited a couple of times. |
I tried in the changeset page provided by @SomeoneElseOSM, if I click on the changeset object the geometry highlight doesn't load (bbox for changeset, line for way, circle for node). GET http://api06.dev.openstreetmap.org/api/0.6/way/NaN/full 404 (Not Found) |
I agree that not enough attention is drawn to a changeset comment. Maybe it should be emphasized somehow. |
... and another comment on the "welcome" screen. Here's what I see with browser font sizes set the way that they normally are for me. "Welcome" is way too big and there seems to be no way of getting rid of it if you haven't signed up, and the layer display is bigger than the height of the screen. The latter may be something best dealt with in a future revision - would it be possible to not display the graphical stuff in the layer menu if it doesn't fit on the screen? |
Look people, you've had weeks to comment on this. It's merged now, and commenting more here is not going to achieve anything. |
Most people have only become aware of the changes now that you pushed them to production. I must say I'm really unhappy with the new history browser. The old one just worked. The new one:
All these things were working perfectly in the old history viewer. |
This pull request tracks work on a redesign of the header, sidebar, and main page map based on the presentation (video, slides, mock ups) at State of the Map Birmingham.
Major components of the redesign include a new help page; new about page; moving navigation elements out of the sidebar; moving search results, export controls, changeset history, and feature details into the sidebar; and removal of the sidebar from "interior" pages such as the user profile and diaries.
Test Deploy
TODO List